Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[14.0][IMP]l10n_it_delivery_note: split move lines based on dn #4375

Open
wants to merge 1 commit into
base: 14.0
Choose a base branch
from
Open

[14.0][IMP]l10n_it_delivery_note: split move lines based on dn #4375

wants to merge 1 commit into from

Conversation

PicchiSeba
Copy link
Contributor

@PicchiSeba PicchiSeba commented Sep 16, 2024 •
edited by francesco-ooops
Loading

Continuazione di #4324 (mergiata per errore)

@OCA-git-bot
Copy link
Contributor

Hi @MarcoCalcagni, @aleuffre, @renda-dev,
some modules you are maintaining are being modified, check this out!

@sergiocorato
Copy link
Contributor

Sto verificando se riesco a risolvere #4324 (comment)

@sergiocorato
Copy link
Contributor

@PicchiSeba PyTech-SRL#3

@PicchiSeba PicchiSeba force-pushed the 14.0-fix-4324 branch 3 times, most recently from d133f0c to 65ec93b Compare October 4, 2024 13:08
@PicchiSeba PicchiSeba marked this pull request as ready for review October 4, 2024 13:08
sergiocorato
sergiocorato approved these changes Oct 7, 2024
View reviewed changes
Copy link
Contributor

@sergiocorato sergiocorato left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Per me è ok, attendiamo review funzionali

@francesco-ooops
Copy link
Contributor

@sergiocorato sto testando 👍

francesco-ooops
francesco-ooops requested changes Oct 7, 2024
View reviewed changes
Copy link
Contributor

@francesco-ooops francesco-ooops left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

SO, product 1, qty 2
Consegno qty 1, backorder, creo e valido DDT1 > Fatturo da ordine e valido INV001
Consegno qty 1, creo e valido DDT2 > Fatturo INV002
Annullo INV002 > torno nell'ordine e fatturo nuovamente

INV003 è relazionato a DDT1 e DDT2

Risultato atteso: INV003 è relazionato a DDT2

@francesco-ooops francesco-ooops mentioned this pull request Oct 7, 2024
Open
2 tasks
@francesco-ooops francesco-ooops linked an issue Oct 7, 2024 that may be closed by this pull request
l10n_it_delivery_note: creando una fattura da più ddt le righe non vengono generate basandosi sulle righe dei ddt #3979
Open
2 tasks
@francesco-ooops francesco-ooops mentioned this pull request Oct 7, 2024
Open
@francesco-ooops
Copy link
Contributor

@sergiocorato @PicchiSeba

francesco-ooops
francesco-ooops requested changes Oct 8, 2024
View reviewed changes
Copy link
Contributor

@francesco-ooops francesco-ooops left a comment •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

SO, product 1, qty 2
Consegno qty 1, backorder, creo e valido DDT1 > Fatturo e valido INV001
Consegno qty 1, creo e valido DDT2 > Fatturo INV002
Annullo INV002 > torno nell'ordine e fatturo nuovamente

INV003 è relazionato a DDT1

Risultato atteso: INV003 è relazionato a DDT2

francesco-ooops
francesco-ooops requested changes Oct 15, 2024
View reviewed changes
Copy link
Contributor

@francesco-ooops francesco-ooops left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

SO, product 1, qty 2
Consegno qty 1, backorder, creo e valido DDT1 > Fatturo e valido INV001
Consegno qty 1, creo e valido DDT2 > Fatturo INV002
Annullo INV002

CURRENT BEHAVIOR

Lo stato di DDT1 è "validato" e non "fatturato"

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sergiocorato sergiocorato sergiocorato approved these changes

@odooNextev odooNextev Awaiting requested review from odooNextev

@francesco-ooops francesco-ooops Awaiting requested review from francesco-ooops

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
5 participants
@PicchiSeba @OCA-git-bot @sergiocorato @francesco-ooops @aleuffre